Skip to content

Conversation

@mijovic
Copy link
Collaborator

@mijovic mijovic commented Oct 4, 2025

This PR:

  • applies 100% penalty on unused gas
  • Fixes tests to fit charging gas limit


uint256 private constant REVERT_REASON_MAX_LEN = 2048;
uint256 private constant PENALTY_PERCENT = 10;
uint256 private constant PENALTY_PERCENT = 100;
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only change required to charge gas full limit specified in userOp

@mijovic mijovic force-pushed the v0.7-monad-gas-limit branch from 1304944 to 3f88c50 Compare October 4, 2025 19:43
@polus-arcticus
Copy link

noting that removal of https://github.com/eth-infinitism/account-abstraction/blob/cc3893bcaf2272c163ce89d5eb9eadb8e6b52db7/contracts/core/EntryPoint.sol#L47 would be relevant for this but likely occurred in a different PR . with no refunds this threshold should be zero.

https://github.com/eth-infinitism/account-abstraction/blob/cc3893bcaf2272c163ce89d5eb9eadb8e6b52db7/contracts/core/EntryPoint.sol#L37

i wanted to justify bumping this due higher costs for several operations. However no cold loads should happen and the operations being performed here dont seem to have different gas costs associated with them so 10k seems fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants